ENH: Migrate recipe to v1 format for rattler-build - #200
Conversation
0977a75 to
e022f80
Compare
|
@conda-forge-admin, please rerender |
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipe/recipe.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/28073600391. Examine the logs at this URL for more detail. |
…2026.06.03.23.09.2
|
I think this migration is currently dropping CUDA 12.0/12.1 support for linux-aarch64, which we want to continue supporting on the 12.9 branch. In the 12.9 branch, In this PR, cufile_supported: aarch64and the generated Since CUDA 12.0/12.1 aarch64 packages are still intended to be supported for the 12.9 branch, I think |
|
@kkraus14 nice find! My mistake was assuming that these two are equivalent: ### Current change
# recipe.yaml
cufile_supported: ${{ False if aarch64 else True }}
### Previous support
# recipe.yaml
cufile_supported: ${{ cufile_supported }}
# conda_build_config.yaml
cufile_supported:
- True
- False # [aarch64]Just pushed the fix for this :) |
|
Hm, CI didn't like some of those changes; I'll see what's going on in a bit |
|
In a nutshell, {
"cuda-bindings-12.9.7-py313h306daec_0.conda": {
"recipe": {
"build_platform": "linux-64",
"c_compiler": "gcc",
"c_compiler_version": "14",
"c_stdlib": "sysroot",
"c_stdlib_version": "2.17",
"channel_targets": "conda-forge main",
"cufile_supported": "False",
"cxx_compiler": "gxx",
"cxx_compiler_version": "14",
"numpy": "2",
"python": "3.13.* *_cp313",
"target_platform": "linux-aarch64"
}
}
}It should certainly be older because of this line. I suppose the work in this PR has uncovered a bug that's been hiding for a while, and pinning the compiler toolchain to an older version fixes it. |
This reverts commit e93ed2c.
|
It reached a passing state 🥳 |
|
Hi! This is the friendly automated conda-forge-linting service. I failed to even lint the recipe, probably because of a conda-smithy bug 😢. This likely indicates a problem in your This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/28112100995. Examine the logs at this URL for more detail. |
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipe/recipe.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/28142009176. Examine the logs at this URL for more detail. |
|
The linter's complaint of Also looks like conda-forge is migrating from Azure to GHA (progress can be tracked here). Some repos have already adopted this change, and I could do that here with this change (I ran diff --git a/conda-forge.yml b/conda-forge.yml
index 6e2b9a8..265aa72 100644
--- a/conda-forge.yml
+++ b/conda-forge.yml
@@ -1,7 +1,3 @@
-azure:
- free_disk_space: true
- settings_linux:
- swapfile_size: 10GiB
bot:
abi_migration_branches:
- '11.8'
@@ -23,3 +19,9 @@ noarch_platforms:
- linux_64
- linux_aarch64
- win_64
+workflow_settings:
+ free_disk_space:
+ - value: "quick"
+ os: win
+ - value: "skip"
+ os: linux |
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)Backporting this change